Skip to content

Conversation

@rickhanlonii
Copy link
Member

Overview

Adds back a unstable_strictMode option to the root which allows opt-ing into strict mode at the root.

This keeps the levels implementation, which is used for the strict effects GK, but collapses it to a single option until theres sufficient demand for opting into different levels at the root.

@rickhanlonii rickhanlonii requested a review from bvaughn May 4, 2021 18:41
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels May 4, 2021
mode |= StrictEffectsMode;
}
}
} else if (enableStrictEffects && createRootStrictEffectsByDefault) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can delete this after some internal clean ups.

@sizebot
Copy link

sizebot commented May 4, 2021

Comparing: d1542de...b847f1f

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 122.76 kB 122.76 kB = 39.42 kB 39.42 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js +0.04% 129.24 kB 129.29 kB +0.07% 41.47 kB 41.50 kB
facebook-www/ReactDOM-prod.classic.js +0.06% 406.95 kB 407.18 kB +0.05% 75.30 kB 75.34 kB
facebook-www/ReactDOM-prod.modern.js +0.02% 394.93 kB 395.01 kB +0.05% 73.37 kB 73.41 kB
facebook-www/ReactDOMForked-prod.classic.js +0.06% 406.95 kB 407.18 kB +0.05% 75.30 kB 75.34 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against b847f1f

@rickhanlonii rickhanlonii force-pushed the rh-strict-root-option branch from 304a5b2 to b4e9696 Compare May 4, 2021 18:56
@rickhanlonii rickhanlonii merged commit e9a4a44 into facebook:master May 4, 2021
@rickhanlonii rickhanlonii deleted the rh-strict-root-option branch May 4, 2021 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants